Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for Quarto integration #746

Merged
merged 20 commits into from
Oct 11, 2023
Merged

Add support for Quarto integration #746

merged 20 commits into from
Oct 11, 2023

Conversation

wch
Copy link
Collaborator

@wch wch commented Oct 4, 2023

This PR adds support for using Shiny in a Quarto document.

It also requires posit-dev/py-htmltools#61

shiny/_app.py Outdated Show resolved Hide resolved
shiny/_main.py Outdated Show resolved Hide resolved
@@ -89,7 +97,7 @@ def server(input: Inputs, output: Outputs, session: Session):

def __init__(
self,
ui: Tag | TagList | Callable[[Request], Tag | TagList],
ui: Tag | TagList | Callable[[Request], Tag | TagList] | Path,
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jcheng5 What do you think about taking a Path object to indicate that it should read the UI from a file?

@wch wch marked this pull request as ready for review October 11, 2023 21:28
@wch wch merged commit efaf8aa into main Oct 11, 2023
24 checks passed
@wch wch deleted the quarto-ext branch October 11, 2023 21:40
schloerke added a commit that referenced this pull request Oct 13, 2023
* main:
  Remove leading newline
  Stop testing on Python 3.12
  Update changelog
  Check that file is autogenerated before overwriting
  Run tests on Python 3.12
  Add support for Quarto integration (#746)
  Revert inadvertent whitespace change
  Update shinylive version for docs (#759)
  Self outputting renderers (#747)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants